Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "enh: breadcrumbs icons are now more visually descriptive" #43625

Conversation

emoral435
Copy link
Contributor

Reverts #43604

We still need to get nextcloud-libraries/nextcloud-vue#5216 merged first before we get the previous PR merged 🥹

Without having the newest NcVue library version, it will introduce a regression as such:

firefox_0G03fcmqBJ

Once the new library gets merged, will add the changes back!

@skjnldsv
Copy link
Member

We can also just fast forward to a new NC/Vue release. This is 29 as alpha state, that happens :)

@emoral435
Copy link
Contributor Author

Ah, okay! Truthfully, I was a bit nervous about making the breaking visual change on the developer instance, but the buttons still work, and having to submit the same PR again would be a lot. Closing PR unless otherwise told to open again.

@emoral435 emoral435 closed this Feb 19, 2024
@emoral435 emoral435 deleted the revert-43604-enh/breadcrumbs/43569/apply-breadcrumbs-logic-from-personal-files-to-all-files-view branch February 19, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants